-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
front: 잘못된 정보 표현 수정 및 사용자 경험 개선 #320
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
minseok-oh
approved these changes
Aug 27, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍 고생하셨습니다!
HiiWee
pushed a commit
that referenced
this pull request
Aug 28, 2024
* feat: 경매 가격 경신 로직 수정 * fix: 경매 가격 하락 로직 수정 * feat: 재고 수량을 폴링하는 로직 추가 (시연용) * feat: 정말로 로그아웃할까요? 문구 추가 * feat: 경매 입찰 성공을 시각적으로 표시. 및 따닥 방지 * style: list 미리보기 이미지 수정 * style: 목록에서는 현재가격을 추정할 수 없으므로 '시작 가격'으로 문구 변경 * feat: 타이머를 통해 다음 가격 변동 시점을 노출 * feat: 회원가입 시 아이디, 패스워드 규격을 사용자에게 안내한다. * feat: API 요청 실패 시, 일부 케이스에서 서버가 전달한 메세지를 표시 * feat: 활성항목을 리스트에 표시 * refactor: 진행 중인 경매에 색상을 표시한다. * feat: 회원가입 성공시 성공 멘트 표시 * refactor: 거래내역에서 생성일과 수정일을 표시하지 않습니다. (서버에서 생성하지 않음) * refactor: 판매자 가입 기능 제거 * fix: 경매 상세페이지에서 가격을 갱신하는 타이머 오류 수정 * fix: 종료된 경매에서 입찰버튼이 활성화되던 문제 수정 --------- Co-authored-by: MinSeok Oh <[email protected]>
yudonggeun
pushed a commit
that referenced
this pull request
Aug 28, 2024
* remove: test 모듈 삭제 - 기존 test 모듈에서 공통으로 참조하고 있는 context, fixture들을 중복을 상관하지 않고 옮긴다. * fix: 모듈 의존성 및 테스트 정리 * docs: api docs 업데이트 * feat: dev 브랜치 918b31d 커밋 적용 * feat: 1dbe550, a2771af 커밋 재적용 * feat: dev 브랜치 63eebfb 커밋 적용 * �front: 잘못된 정보 표현 수정 및 사용자 경험 개선 (#320) * feat: 경매 가격 경신 로직 수정 * fix: 경매 가격 하락 로직 수정 * feat: 재고 수량을 폴링하는 로직 추가 (시연용) * feat: 정말로 로그아웃할까요? 문구 추가 * feat: 경매 입찰 성공을 시각적으로 표시. 및 따닥 방지 * style: list 미리보기 이미지 수정 * style: 목록에서는 현재가격을 추정할 수 없으므로 '시작 가격'으로 문구 변경 * feat: 타이머를 통해 다음 가격 변동 시점을 노출 * feat: 회원가입 시 아이디, 패스워드 규격을 사용자에게 안내한다. * feat: API 요청 실패 시, 일부 케이스에서 서버가 전달한 메세지를 표시 * feat: 활성항목을 리스트에 표시 * refactor: 진행 중인 경매에 색상을 표시한다. * feat: 회원가입 성공시 성공 멘트 표시 * refactor: 거래내역에서 생성일과 수정일을 표시하지 않습니다. (서버에서 생성하지 않음) * refactor: 판매자 가입 기능 제거 * fix: 경매 상세페이지에서 가격을 갱신하는 타이머 오류 수정 * fix: 종료된 경매에서 입찰버튼이 활성화되던 문제 수정 --------- Co-authored-by: MinSeok Oh <[email protected]> * chore: scp로 api, consumer 모듈 말아서 전송하도록 스크립트 수정 * chore: dev2 배포 스크립트 추가 * feat: 정적 리소스 필터링 컨트롤러도 그냥 pass하도록 설정 추가 * �front: 초 단위 가격 갱신 적용 및 재고 부족 시 응답이 200으로 오는 부분에 맞게 수정 (#325) * feat: 경매 상품의 초단위 가격 하락 지원 * refactor: 경매 재고 부족시 200응답 + A012 조합을 가져와 Fail 상황으로 처리한다. * feat: ver2를 위한 로직 추가 --------- Co-authored-by: HyeonSik Choi <[email protected]> Co-authored-by: MinSeok Oh <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📄 Summary
1순위 (크리티컬함)
2순위 (서버 로직 입장에서는 무리가 없지만...)
3순위 (편의)
🙋🏻 More
close #307